Skip to content

Conversation

@garybadwal
Copy link
Contributor

@garybadwal garybadwal commented Oct 18, 2025

Related Issues

Proposed Changes:

  • adds a new delete_all_documents() along with its asynchronous counterpart
  • includes both simple deletion and index recreation modes

How did you test it?

  • Unit tests, integration tests, and manual verification.
  • Added new tests:
    • test_delete_all_documents_no_index_recreation(): Verifies simple deletion functionality.
    • test_delete_all_documents_index_recreation(): Tests index recreation with configuration preservation.
    • test_delete_all_documents_async(): Async version of simple deletion test.
    • test_delete_all_documents_index_recreation(): Async version of index recreation test.

Checklist

@garybadwal garybadwal requested a review from a team as a code owner October 18, 2025 06:28
@garybadwal garybadwal requested review from julian-risch and removed request for a team October 18, 2025 06:28
@github-actions github-actions bot added integration:chroma type:documentation Improvements or additions to documentation labels Oct 18, 2025
@garybadwal
Copy link
Contributor Author

garybadwal commented Oct 18, 2025

Hi @davidsbatista, I’ve added the delete_all_documents functionality for ChromaDBDocumentStore. Could you please review it?

@garybadwal garybadwal changed the title feat: adding delete_all_docs to Qdrant document store feat: adding delete_all_docs to ChromaDB document store Oct 18, 2025
@garybadwal
Copy link
Contributor Author

Hi @julian-risch, Hope you are doing great, can you give a review to this PR. I will be happy to contribute in this and will be make fixes or changes that you will suggest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:chroma type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add delete_all_documents() operation to ChromaDocumentStore

1 participant